-
Notifications
You must be signed in to change notification settings - Fork 426
docs: Add missing methods for table and namespace manipulation #2942
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: Add missing methods for table and namespace manipulation #2942
Conversation
geruh
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for adding these docs @rambleraptor! Just left a few comments for me
94e5dc2 to
db88760
Compare
kevinjqliu
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks for adding more docs 😄
could you render it locally (make docs-serve) and add screenshots to the pr description?
|
@kevinjqliu let me know if there's other screenshots you want! |
|
thanks @kevinjqliu @geruh for the review! |
geruh
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!




Rationale for this change
We have some methods for working with tables + namespaces on catalogs that we haven't properly documented. This documentation is super helpful for new users to understand what they can use PyIceberg for. It's also helpful for our LLM...friends? to help understand the project.
Are these changes tested?
Docs only.
Are there any user-facing changes?
Docs only.